app/models/agent.rb

@@ -195,7 +195,6 @@ class Agent < ActiveRecord::Base

                   end
              

                 
              

                   def log(message, options = {})
              

                -    puts "Agent##{id}: #{message}" unless Rails.env.test?
              

                     AgentLog.log_for_agent(self, message, options)
              

                   end
              

                 
              

app/models/agent_log.rb

@@ -15,6 +15,8 @@ class AgentLog < ActiveRecord::Base

                   before_save :truncate_message
              

                 
              

                   def self.log_for_agent(agent, message, options = {})
              

                +    puts "Agent##{agent.id}: #{message}" unless Rails.env.test?
              

                +
              

                     log = agent.logs.create! options.merge(:message => message)
              

                     if agent.logs.count > log_length
              

                       oldest_id_to_keep = agent.logs.limit(1).offset(log_length - 1).pluck("agent_logs.id")
              
@@ -34,6 +36,7 @@ class AgentLog < ActiveRecord::Base

                 
              

                   def scrub_message
              

                     if message_changed?
              

                +      self.message = message.inspect unless message.is_a?(String)
              

                       self.message.scrub!{ |bytes| "<#{bytes.unpack('H*')[0]}>" }
              

                     end
              

                     true
              

spec/models/agent_log_spec.rb

@@ -90,6 +90,11 @@ describe AgentLog do

                       AgentLog.log_for_agent(agents(:jane_website_agent), "some message", :level => 4, :outbound_event => events(:jane_website_agent_event))
              

                       expect(agents(:jane_website_agent).reload.last_error_log_at.to_i).to be_within(2).of(Time.now.to_i)
              

                     end
              

                +
              

                +    it "accepts objects as well as strings" do
              

                +      log = AgentLog.log_for_agent(agents(:jane_website_agent), events(:bob_website_agent_event).payload)
              

                +      expect(log.message).to include('"title"=>"foo"')
              

                +    end
              

                   end
              

                 
              

                   describe "#log_length" do